Conversation
There was a problem hiding this comment.
Комментарии к самому стеку и про ветки всё те же
| return 0; | ||
| } | ||
|
|
||
| void infixToPostfix(const char* infix, char* postfix) |
There was a problem hiding this comment.
Если уж Вы выделяете память на postfix извне, будьте добры описать, сколько её надо
| while (isdigit(infix[i])) { | ||
| postfix[j++] = infix[i++]; | ||
| } |
There was a problem hiding this comment.
Интересное решение, кажется, даже числа заработают. Хотя просили только цифры
| return 0; | ||
| } | ||
|
|
||
| void infixToPostfix(const char* infix, char* postfix) |
There was a problem hiding this comment.
Кажется, здесь где-то не хватает обработки небалансных скобок
| int i = 0; | ||
| int j = 0; |
There was a problem hiding this comment.
Переменные без внятного названия на верхнем уровне --- плохая идея. Что-нибудь типа prefixIdx и postfixIdx было бы сильно лучше
| int topVal = 0; | ||
| int val = 0; |
There was a problem hiding this comment.
Во-первых, по-моему это должна быть одна переменная. А во-вторвых, лучше переменную объявлять по месту использования. Мало ли что там будет.
2 participants
No description provided.